Cape Town | 26-Sep-ITP | Raees Stevens | Sprint 1 | Form Controls - #1518
RaeesStevens wants to merge 45 commits into
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This comment has been minimized.
This comment has been minimized.
|
Thank you for your submission. The following are a quick summary of the fixes needed before this PR can be merged. In Form-Controls/index.html, please remove the extra "X" option from the size dropdown so it matches the required 6 options, and add a pattern attribute (pattern=".[^\s].[^\s].*") to the name input to enforce at least two non-space characters. Additionally, improve the form's semantics and accessibility by moving the size dropdown and submit button outside of the color , and update the size label to use for="size". Finally, please fix the typo in Form-Controls/README.md on line 43 ("JavaScrip" -> "JavaScript"). Once these four items are updated, this will be good to go. |
RaeesStevens
left a comment
There was a problem hiding this comment.
Updated with changes from review.
Thank you. I have made changes and submitted for review. Please note update to the title of the PR to 2026. |

Cape Town | 25-Sep-ITP | Raees Stevens | Sprint 1 | Form Controls
Self checklist
Task code
CYF-1004
Changelist
Created a form
Added inputs and labels for customer name and email, t-shirt colours and sizes.
Basic form styling using CSS.